Skip to content

Add OWNERS file #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 17, 2018
Merged

Add OWNERS file #45

merged 1 commit into from
Aug 17, 2018

Conversation

spiffxp
Copy link
Member

@spiffxp spiffxp commented Aug 14, 2018

Populated based upon users who have previously merged code in this repo

Fixes #44

@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 14, 2018
@spiffxp
Copy link
Member Author

spiffxp commented Aug 14, 2018

/cc @apelisse @mengqiy @thockin @dashpole

@thockin
Copy link
Member

thockin commented Aug 14, 2018

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 14, 2018
@deads2k
Copy link
Contributor

deads2k commented Aug 14, 2018

/hold

Can we add a non-googler to this review/approver.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Aug 14, 2018
@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 14, 2018
@spiffxp
Copy link
Member Author

spiffxp commented Aug 14, 2018

@deads2k ack, I added you

FWIW I'm just trying to get something in place that represents the state of who has merged in the past. I am in highly in favor of this list representing interested owners, and would encourage you all to revise this as needed

@spiffxp
Copy link
Member Author

spiffxp commented Aug 14, 2018

(this repo isn't yet setup to use /approve precisely because it lacks OWNERS, so if y'all are ok with it I can manually merge)

@lavalamp
Copy link
Member

I'd like to be somewhere in the file because clock is my baby from way back. I am fine with you manually merging.

Populated based upon users who have previously merged code in this repo

Added lavalamp based on review feedback
@spiffxp
Copy link
Member Author

spiffxp commented Aug 17, 2018

Added @lavalamp

Will manually merge and turn on /approve automation for this repo. We can then iterate on this as needed

@spiffxp spiffxp merged commit 982821e into kubernetes:master Aug 17, 2018
@spiffxp spiffxp deleted the add-owners-file branch August 17, 2018 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants